Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MyECLPay #611

Draft
wants to merge 68 commits into
base: main
Choose a base branch
from
Draft

MyECLPay #611

wants to merge 68 commits into from

Conversation

armanddidierjean
Copy link
Member

Description

Please explain the changes you made here.

Checklist

  • Created tests which fail without the change (if possible)
  • All tests passing
  • Extended the documentation, if necessary

@armanddidierjean armanddidierjean force-pushed the myeclpay branch 2 times, most recently from 47f1137 to 1777ada Compare November 17, 2024 08:04
Copy link

codecov bot commented Nov 17, 2024

Codecov Report

Attention: Patch coverage is 81.10860% with 167 lines in your changes missing coverage. Please review.

Project coverage is 81.05%. Comparing base (1d1de79) to head (43e283a).

Files with missing lines Patch % Lines
app/core/myeclpay/endpoints_myeclpay.py 70.39% 135 Missing ⚠️
app/core/myeclpay/cruds_myeclpay.py 86.61% 17 Missing ⚠️
app/core/myeclpay/utils_myeclpay.py 67.50% 13 Missing ⚠️
app/core/payment/cruds_payment.py 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #611      +/-   ##
==========================================
- Coverage   81.06%   81.05%   -0.01%     
==========================================
  Files         134      140       +6     
  Lines       10319    11191     +872     
==========================================
+ Hits         8365     9071     +706     
- Misses       1954     2120     +166     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Marc-Andrieu Marc-Andrieu mentioned this pull request Dec 25, 2024
4 tasks
foucblg pushed a commit that referenced this pull request Jan 6, 2025
> [!IMPORTANT]
> We really need to release new versions **soon**, as we have not for
several months and there's some time needed to test in alpha before
prod.

This goes hand in hand with the **major version bump** [for
Titan](../../Titan/pull/462).

Both PRs are ready, and _seemingly_ waiting for some more **breaking
changes** to be merged, including (by likelihood):

- [ ] Allow students from other Schools to have a MyECL account: #641 
- [ ] MyECLPay: #611 
- [ ] "_Centrale Mega Meme_" Meme module: [no PR yet] [see
branch](../tree/cmm)
- [x] External Notifications: #613
armanddidierjean and others added 8 commits January 15, 2025 01:24
### Description

Please explain the changes you made here.

### Checklist

- [ ] Created tests which fail without the change (if possible)
- [ ] All tests passing
- [ ] Extended the documentation, if necessary
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request module payment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants